GtkSearchEngine: Remove unused code

Nothing ever uses or emits the hits-subtracted signal.
Nautilus dropped it a few years ago. Time to follow suit.

Use infos from search engine instead of getting them again

When the search engine provides hits with GFileInfo, use that
to add the hits to the model directly, without going through
another round of async get_info calls.

To do this, we add a batched variant of the
_gtk_file_system_model_update_file call that takes lists of
GFiles and GFileInfos. Again, we can avoid repeated resorting
that happens when the files are updated individually.

Fix an oversight in the previous commit

We lost the line marking the search model as non-empty, causing
us to show the 'no results' page despite there being results.

Request the right attributes in the simple search engine

This change makes it so that the file info passed along
from the simple search engine has all the attributes that
the file system model wants.

search engine: Pass file infos along for hits

file chooser: Use batched insertion for search results

This helps avoiding excess work when procesing many search
results, as typically happens with short search strings
and recursive search.

https://bugzilla.gnome.org/show_bug.cgi?id=751169

file system model: Add batched insertion

Add a batched version of gtk_file_system_model_add_and_query_file
that takes a list of files and avoids resorting the model for each
individual insertion. The querying is still done one-file-at-a-time,
so more optimization is certainly possible.

Revert "window: Reinstate logic for should_use_csd()"

This reverts commit f043fd532dc66a7abb1d4ea57c666655330cc429.

The whole point of having can_use_csd() return TRUE when GTK_CSD is
unset is to use it inside enable_csd().

https://bugzilla.gnome.org/show_bug.cgi?id=751140

window: Reinstate logic for should_use_csd()

The old should_use_csd() function would return FALSE if the GTK_CSD
environment variable is unset; the change in commit c5e5ee67490e7e7
made it return TRUE if GTK_CSD is unset. This has a cascade effect
on the window size, which causes invalid rectangles to bubble down
to Pixman.

https://bugzilla.gnome.org/show_bug.cgi?id=751140

gtkdialog: Remove 2px border

messagedialog: remove 2px border

filechooserdialog: Remove space between window and content

Was removed in 232d6f84f0495e9effedc16d700fc13f5a6e302.

Adwaita: indentation

- lapo was shouting at me.

Adwaita: volume button and scale button padding

I was assuming that GtkVolumeButton and GtkScaleButton could only
contain a icon, now they get the image-button padding by default,
but if the text-button class is applied it is not overridden.

Adwaita: scale popup quick fix

Adwaita: markup cleanup

- move touch selection selectors to its own section
- use scss style markup for .button common elsewhere
  (more compact and legible)

HC: touch text selection styling

- sync with Adwaita
- try to avoid using image assets

https://bugzilla.gnome.org/show_bug.cgi?id=750883

Fix return value error in _gtk_window_get_popover_parent

Cosmetic change

GtkPlacesSidebar: Fix rtl appearance

Apply the necessary directional variants for asymmetric
css padding.

https://bugzilla.gnome.org/show_bug.cgi?id=751084

overlay: remove gtk_overlay_add_pass_through_overlay()

The API to access this functionality will be the setter we just added in
the previous commit.

https://bugzilla.gnome.org/show_bug.cgi?id=750568

overlay: add setters and getters for pass-through child property

This will make the API easier to use from bindings too.

https://bugzilla.gnome.org/show_bug.cgi?id=750568

docs: add new GtkOverlay methods to gtk3-sections.txt

https://bugzilla.gnome.org/show_bug.cgi?id=750568

configure: test have_docbook_style != yes, not have_docbook_dtd

file chooser: Drop unused variables

file chooser: Use a revealer

Use a revealer to manage the visibility of the header area where
we show the pathbar, the location entry or search. This is a bit
smoother, and makes search more similar to a search bar.

texthandle: Avoid double rendering of background/frame

gtk_render_handle() already renders background/frame itself, avoid
doing this twice.

gtkmain: Ignore grab for events in child popovers

Popovers may be spawn when there's GTK+ grabs somewhere else (eg.
text selection popover/handles in an entry in a modal popover). When
this happens, events go to the grab widget (in this case the modal
popover) and are effectively ignored by the event widget, even though
it's can be conceptually a child of the grab widget.

To get away with this, tweak a bit gtk_main_do_event(), so events going
to popovers that are related to grab_widget or a child of it are received,
as it would happen with regular children of grab_widget.

https://bugzilla.gnome.org/show_bug.cgi?id=750993

window: Add concept of popover "parent"

This will be the widget that the popover relates to (::pointing-to in
GtkPopover, ::parent in GtkTextHandle).

Additional API to check the popover/parent relationship between widgets
has been added, which will be useful wherever this is necessary in a
generic manner.

https://bugzilla.gnome.org/show_bug.cgi?id=750993

Fix return value error in gdk_window_set_pass_through

https://bugzilla.gnome.org/show_bug.cgi?id=751099

wayland: Always request transient commit when mapping subsurface

A subsurface positioning operation only takes effect when the parent
surfaces state is applied. If a subsurface is mapped and positioned, but
the parent surface state is not immediately committed, the relative
position of the subsurface is undefined and may be placed incorrectly.

To avoid this undefined state, always request that the parent surface
should be committed after mapping a subsurface so that the position
operation will take effect.

https://bugzilla.gnome.org/show_bug.cgi?id=751098

treeview: always pixelcache widget contents

GtkTreeView has a particularly expensive drawing path. This can cause
issues when part of animated widget sequences. Caching the content while
a model is attached helps reduce the number of full redraws during
exposure greatly.

https://bugzilla.gnome.org/show_bug.cgi?id=751082

pixelcache: allow widgets to always require cached content

Some widgets have very expensive drawing paths. So caching the content
can be useful even when not scrolling.

This can help speed up widgets that are part of animation sequences and
thereby go through spurious expose events.

https://bugzilla.gnome.org/show_bug.cgi?id=751082

Updated Hebrew translation

gdkwindow: avoid updating background pattern if it matches previous

Background patterns are often updated when style changes. In many cases,
the new pattern will match the previous. We can optimize out the
invalidation that will occur upon resetting the same pattern.

gtkplacessidebar: improve variable name

gtkplacessidebar: Remove some old comments

We didn't made up our minds until now from some of them, and for
the others no progress has been made or bugs reported, so probably
they are fine.

gtkplacesidebar: use GtkListBox

We were using GTkTreeView in a simple list. Also, as we know,
GtkCellRenderers are not the best way to theme and manipulate
widgets.

So instead use a GtkListBox to modernize the GtkPlacesSidebar,
and in the way clean up some parts of the code (like headings)
which were not used anymore.
Also we don't use a model anymore, since the data is simple
enough to manage it in a subclass of the row itself.

theme: decrease list row transition

It was somewhat annoying given the "trace" of it. Decrease
the time to avoid too much "trace"

gtkfilechooserwidget: show drop hints on gtkplacessidebar

Now with the API addition on gtkplacessidebar, we can show some
hints for the drop targets.

https://bugzilla.gnome.org/show_bug.cgi?id=747793

gtkplacessidebar: make insensitive invalid drop targets

https://bugzilla.gnome.org/show_bug.cgi?id=747793

gtkplacessidebar: add new bookmark icon

Following design mockups

https://bugzilla.gnome.org/show_bug.cgi?id=747793

gtkplacesidebar: remove obsolete drop state

We didn't use those for some time now.

gtkplacessidebar: add API for show drop hints

It is convenient to allow applications to show all the drop
targets at once. This improves the user experience with drag
an drop.

The new API allows the application to set the gtkplacessidebar
in a mode where invalid drop targets are insensitive and it
adds a "new bookmark" row. This mode is intended to be set
when the application is aware of a dnd operation and needs to
be stopped kwhen the application is aware that dnd operation
was cancelled or ended in a different part than gtkplacesisdebar.

The context parameter is unused in this patch, but will be
used in next patches when the sidebar will use a GtkListBox.
The reason of being unused now is just convenience.

https://bugzilla.gnome.org/show_bug.cgi?id=747793

gtkplacessidebar: new bookmark row only at first position

Following the new mockups, put the new bookmark row always
in the first position, so it's easier to drop an item.

https://bugzilla.gnome.org/show_bug.cgi?id=747793

Deal with events from wrong display

GtkInspector is opening a separate display connection, which makes
it more likely that gtk_get_current_event() returns an event from
the "wrong" display.

GtkMenuButton: Make programmatic toggling work again

This was broken in 0796d7b6ff9393746d.

https://bugzilla.gnome.org/show_bug.cgi?id=751018

GtkMenuButton: Avoid direct button access

We were getting ourselves in trouble by casting touch events
to GdkEventButton and poking directly at their internals. Instead,
use GdkEvent API to get what we need.

This fixes a crash when using the gear menu in epiphany with
touch. The same crash also occurred in testmenubutton.

gtkoverlay: Fix issues with remove

The iteration now progresses past a delete, so make sure we iterate
safely. Also, don't chain up if we removed a child.

testoverlay: Fix indentation

testoverlay: Add test for overlay z ordering

https://bugzilla.gnome.org/show_bug.cgi?id=750568

https://bugs.freedesktop.org/show_bug.cgi?id=90917

testoverlay: Fix test by using add_pass_through_overlay

https://bugzilla.gnome.org/show_bug.cgi?id=750568

https://bugs.freedesktop.org/show_bug.cgi?id=90917

GtkOverlay: Add support for input pass-through overlays

For these widgets we set pass-through on the child window so that
input over these widgets (that are not on a child input window) goes
to the window below the overlay.

https://bugzilla.gnome.org/show_bug.cgi?id=750568

https://bugs.freedesktop.org/show_bug.cgi?id=90917

overlay: Add reorder_overlay()

This allows you to control the z-ordering of overlay children

https://bugzilla.gnome.org/show_bug.cgi?id=750568

https://bugs.freedesktop.org/show_bug.cgi?id=90917

gdk: Add gdk_window_set_pass_through

An pass_through window is something you can draw in but does not
affect event handling. Normally if a window has with no event mask set
for a particular event then input events in it go to its parent window
(X11 semantics), whereas if pass_through is enabled the window below
the window will get the event. The later mode is useful when the
window is partially transparent. Note that an pass-through windows can
have child windows that are not pass-through so they can still get events
on some parts.

Semantically, this behaves the same as an regular window with
gdk_window_set_child_input_shapes() called on it (and re-called any
time a child is changed), but its far more efficient and easy to use.

This allows us to fix the testoverlay input stacking test.

https://bugzilla.gnome.org/show_bug.cgi?id=750568

https://bugs.freedesktop.org/show_bug.cgi?id=90917

testoverlay: Add new input stacking test

In this case we have a bunch of interactive main children
of the overlay, and then a centered overlay that contains both
non-interactive (labels) and interactive (entry) widgets.

This shows off a problem where the non-interactive parts (the labels)
steals input from the overlay main children (breaks button click and
hover effects).

https://bugzilla.gnome.org/show_bug.cgi?id=750568

https://bugs.freedesktop.org/show_bug.cgi?id=90917

Updated Spanish translation

Updated Greek translation

stylecontext: Change fallback behavior on state mismatch

For functions that take state flags as an argument we need to special
case the situation where the passed in flags don't match the current
state.

Previously we would create a copy of the style info, change its state
and do the lookup from there.

Now that GtkCssNode has replaced style infos, this doesn't work as well
anymore as copying a GtkCssNode is not possible.
However, unike style infos, GtkCssNodes are instant-apply, so we don't
need to copy anymore, we can just change the state of the node.

This causes some invalidations to be queued, but we can take that
performance hit as this is fallback code.

https://bugzilla.redhat.com/show_bug.cgi?id=1228852

widget-factory: Make the circular button do something

testassistant: Remove padding from the filechooser page

This tests the new assistant api to remove padding from
pages.

GtkAssistant: Add a way to have pages without padding

This can be useful when embedding complex widgets like
a file chooser as a page.

https://bugzilla.gnome.org/show_bug.cgi?id=750631

Add a filechooser-in-assistant example

This is one of the cases where one wants a page without
any padding.

GtkAssistant: Redo page margins

Instead of having padding outside the notebook containing
all pages, put each page in an extra box and add the padding
there. This is in preparation for allowing pages without
padding.

assistant: Drop hardcoded padding

This means the pages themselves will be responsible for setting
suitable padding.

https://bugzilla.gnome.org/show_bug.cgi?id=750631

Fixes to Catalan translation

Only build testfontchooserdialog on linux

We are using this to test pango functionality that
uses fontconfig apis, so this doesn't build on OS X
or Windows anymore.

https://bugzilla.gnome.org/show_bug.cgi?id=750892

color editor: make popups toggleable

The popups on the color editor are somewhat hard to get rid of.
Arrange things so that clicking the same control again dismisses
the current popup.

Don't show a menu on the color editor swatch

There's nothing to customize here, so no need to show a menu.

Add a way to avoid menu on color swatches

Avoid uninitialized variables

clang complains that these may be used uninitialized.

entry: Drop an unused variable

Drop an unused variable

Remove unused variables

Ensure a variable is initialized

clang complained about transient_for being used uninitialized.

Remove an unused variable

broadway: Avoid compiler warnings

clang doesn't like self-assignment.

GtkColorEditor: Remove old todo

Most of this is done, and the rest will not happen because
of this note.

widget-factory: More popover examples

Add an example for text input in a popover. This leads to nested
popovers with touch selection, and does not currently work.

widget-factory: Add horizontally linked entries

It happens in some places, so artists need an example to
make it look good.

Fix 'void function should not return void expression' warning for clang

https://bugzilla.gnome.org/show_bug.cgi?id=750888

Fix return value error in _gtk_text_handle_set_direction

https://bugzilla.gnome.org/show_bug.cgi?id=750888

Redo font map support in GtkFontChooser

We can't add properties to the interface, since it breaks
3rd party implementations of the GtkFontChooser interface.
These exist, for example in gnumeric.

So, instead of a new property, add getter/setter vfuncs.

fontchooser: Don't cause "row-changed" signal in cell data func

The font chooser delays creating the font description from the font face
as long as possible (it's slow). Because we use fixed height mode, we
only have to create font descriptions for rows we are actually going to
show.

This was achieved by looking at the font description column and if it
was NULL, we created a font description and gtk_list_stiore_set() it.
Unfortunately this caused a "row-changed" signal to be emitted and this
emission could happen during the cell data func.
And that caused infinite loops with accessibility when you were unlucky.

This change replaces the NULL font description with an empty one and
instead of setting the correct font description, we
pango_font_description_merge() it in. This way, the list store doesn't
change and no signals are emitted.

https://bugzilla.redhat.com/show_bug.cgi?id=1197267

popover: Avoid hiding modal popovers if another popover is taking focus

Due to popover modality itself, there's quite high chances the popover
stealing focus has been triggered from within, so stay friendly to it.
Hiding the popover here will only hide the grabbing popover too if this
happens.

https://bugzilla.gnome.org/show_bug.cgi?id=750741

window: Avoid rising the window of already shown popovers

This breaks stacking, as popovers are repositioned sort of frequently
(even more after show/hide animations).

https://bugzilla.gnome.org/show_bug.cgi?id=750741

reftests: fix style class syntax in CSS file

Fixes reftests.

https://bugzilla.gnome.org/show_bug.cgi?id=750605

testsuite: add a test for non-square symbolic icons

To verify the previous fix.

https://bugzilla.gnome.org/show_bug.cgi?id=750605

icontheme: don't modify symbolic SVG dimensions when recoloring

When recoloring symbolic SVG, do not modify the original width and
height of the passed-in file; the function later will scale the image
through gdk_pixbuf_new_from_stream_at_scale(), but we should still
use the original size to create the proxy SVG, or the image will
possibly be doubly-resized or blurry.

https://bugzilla.gnome.org/show_bug.cgi?id=750605

recent chooser: Make it expand

It is expected that the list fills the available space.

https://bugzilla.gnome.org/show_bug.cgi?id=750690

entry: Ensure we unset the magnifier widget on ::destroy

Otherwise there's dangling signal handlers which cause warnings
at the time we attempt to disconnect these.

textview: Set handles' mode to "none" if showing the bubble

show_or_hide_handles() tries to disable visibility when the popover is
shown, although it triggers a bit late, and lets the handles flash briefly
if both popover and handles try to show at the same time (eg. when
pressing on the selection of a previously unfocused textview, the handles
were previously hidden, so they try to show again on focus in).

textview: Perform tougher handle visibility checks after hiding the popover

The handles might fall outside the visible area, and shouldn't be shown
then. Just call gtk_text_view_update_handles() which will perform these
checks, and keep the handle conveniently hidden.

This was leading to unexpectedly visible handles (and in the
wrong/previous position, the handle code doesn't relocate the widget
it's about to hide) when "select all" was selected in the popover on
a textview needing scrollbars.
and extending the selection beyond the view above and/or below.

popover: Be lenient wrt visibility of popovers too close to widget borders

The check used to hide the popover if the pointed area fell partly out of
the widget allocation, textviews now can trigger that with text selections
too close to the visible edge, as a small extra area around is now reserved.

The check has been changed to only hide the popover if the pointed area
falls completely outside the widget allocation.

texthandle: Remove knowledge about window borders

We need to expand beyond these if necessary, so don't artificially
limit things here.

window: Allow more space at edges for popovers

We're clamping to the visible window edge, which misplaces text handles.

entry: Set text direction information on text handles

This way the handles point to the right side on the presence of rtl/ltr
text.

textview: set text direction information on text handles

This makes them point to the correct place, given the theme supports
this.

texthandle: Handle ltr/rtl positioning

This setting is per handle, as is dependent on the selected text, rather
than the locale.

Adwaita: support rtl on text handles

Just use the opposite variant for the opposite LTR side.